Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Audit fixes #1395

Open
wants to merge 22 commits into
base: develop-combo-futarchy
Choose a base branch
from
Open

Audit fixes #1395

wants to merge 22 commits into from

Conversation

maltekliemann
Copy link
Contributor

What does it do?

What important points should reviewers know?

Is there something left for follow-up PRs?

What alternative implementations were considered?

Are there relevant PRs or issues?

References

- Clarify `ComboMath` documentation
- Make test `combo_buy_fails_on_amount_out_below_min` more precise
@mergify mergify bot added the s:in-progress The pull requests is currently being worked on label Dec 2, 2024
@maltekliemann maltekliemann requested a review from sea212 as a code owner December 3, 2024 13:20
@maltekliemann maltekliemann added s:review-needed The pull request requires reviews and removed s:in-progress The pull requests is currently being worked on labels Dec 5, 2024
@mergify mergify bot added s:in-progress The pull requests is currently being worked on and removed s:review-needed The pull request requires reviews labels Dec 5, 2024
@maltekliemann maltekliemann added s:review-needed The pull request requires reviews and removed s:in-progress The pull requests is currently being worked on labels Dec 5, 2024
@codecov-commenter
Copy link

codecov-commenter commented Dec 5, 2024

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 91.68975% with 30 lines in your changes missing coverage. Please review.

Project coverage is 93.58%. Comparing base (93f0663) to head (f664336).

Files with missing lines Patch % Lines
zrml/neo-swaps/src/weights.rs 64.55% 28 Missing ⚠️
primitives/src/asset.rs 50.00% 1 Missing ⚠️
...l-tokens/src/types/cryptographic_id_manager/mod.rs 94.11% 1 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@                    Coverage Diff                     @@
##           develop-combo-futarchy    #1395      +/-   ##
==========================================================
+ Coverage                   93.53%   93.58%   +0.04%     
==========================================================
  Files                         178      179       +1     
  Lines                       34547    34572      +25     
==========================================================
+ Hits                        32315    32355      +40     
+ Misses                       2232     2217      -15     
Flag Coverage Δ
tests 93.58% <91.68%> (+0.04%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@mergify mergify bot added s:in-progress The pull requests is currently being worked on and removed s:review-needed The pull request requires reviews labels Dec 9, 2024
commit 4517504
Author: Malte Kliemann <[email protected]>
Date:   Tue Dec 10 18:51:24 2024 +0100

    Benchmark `DecisionMarketOracle`

commit 5832f0b
Author: Malte Kliemann <[email protected]>
Date:   Tue Dec 10 16:55:48 2024 +0100

    Implement `DecisionMarketOracle` using scoreboard

commit e684d47
Author: Malte Kliemann <[email protected]>
Date:   Mon Dec 9 21:24:47 2024 +0100

    Add `BlockNumber` parameter to `update`

commit 5f51378
Author: Malte Kliemann <[email protected]>
Date:   Mon Dec 9 21:17:20 2024 +0100

    Update oracles on each block

commit 2b831fb
Author: Malte Kliemann <[email protected]>
Date:   Mon Dec 9 20:40:26 2024 +0100

    Implement `try_mutate_all`

commit 59cb185
Author: Malte Kliemann <[email protected]>
Date:   Mon Dec 9 18:24:56 2024 +0100

    Use `ProposalStorage` and `MaxProposals`

commit 58afe87
Author: Malte Kliemann <[email protected]>
Date:   Mon Dec 9 17:56:53 2024 +0100

    Add `MaxProposals` value

commit 22068d8
Author: Malte Kliemann <[email protected]>
Date:   Mon Dec 9 14:15:33 2024 +0100

    Add `update` function to `FutarchyOracle` trait
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
s:in-progress The pull requests is currently being worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants